Skip to content

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented May 28, 2025

This switches from rust-lang/simpleinfra/github-actions/static-websites to using GitHub's native actions/deploy-pages to deploy the site. The deploy part of static-websites doesn't seem to be getting updated, and by the discussion at https://rust-lang.zulipchat.com/#narrow/channel/242791-t-infra/topic/Remove.20static-websites.20GitHub.20Action/with/474632056 I get the impression that the infra team would prefer to retire it.

r? marcoieni
Since you seem to have some context here.

I think this will need settings changes before merging. IIUC, static-websites was deploying to a branch, but deploy-pages changes the source. In Settings > Pages > Build and Deployment > Source I think the source needs to be changed to "GitHub Actions", though check my understanding since I am uncertain.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 28, 2025
@ehuss ehuss force-pushed the deploy-gh-pages branch 2 times, most recently from 1234baf to 076a8a4 Compare May 28, 2025 23:37
@Kobzol
Copy link
Member

Kobzol commented May 29, 2025

Yes, I think that the last time we did this we had to switch the source to GitHub Actions in the repository settings.

This switches from rust-lang/simpleinfra/github-actions/static-websites
to using GitHub's native actions/deploy-pages to deploy the site.
The deploy part of static-websites doesn't seem to be getting updated,
and by the discussion at https://rust-lang.zulipchat.com/#narrow/channel/242791-t-infra/topic/Remove.20static-websites.20GitHub.20Action/with/474632056
I get the impression that the infra team would prefer to retire it.
@ehuss ehuss force-pushed the deploy-gh-pages branch from 31ac924 to 3c4612d Compare June 7, 2025 01:33
@marcoieni marcoieni enabled auto-merge June 9, 2025 09:23
@marcoieni marcoieni merged commit d3b780a into rust-lang:master Jun 9, 2025
2 checks passed
@marcoieni marcoieni mentioned this pull request Jun 9, 2025
@ehuss ehuss mentioned this pull request Jun 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants